Skip to content

rust: comments: fix missing Markdown autolinks for URLs#1154

Closed
brambhattabhishek wants to merge 1 commit into
Rust-for-Linux:rust-nextfrom
brambhattabhishek:fix-comment-autolinks
Closed

rust: comments: fix missing Markdown autolinks for URLs#1154
brambhattabhishek wants to merge 1 commit into
Rust-for-Linux:rust-nextfrom
brambhattabhishek:fix-comment-autolinks

Conversation

@brambhattabhishek

@brambhattabhishek brambhattabhishek commented Apr 4, 2025

Copy link
Copy Markdown

rust: comments: fix missing Markdown autolinks for URLs

Some comments in Rust files used plain URLs instead of Markdown
autolinks. This patch wraps those URLs with angle brackets (<...>)
to ensure consistency with other comments and proper rendering in
generated documentation.

No functional code changes are made.

Link: #1153
Signed-off-by: Abhishek Brahmbhatt brambhattabhishek@gmail.com

@brambhattabhishek brambhattabhishek left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please review commit

Fixes Rust-for-Linux#1153 – some comments in Rust files were using plain URLs instead of
Markdown autolink syntax. This change updates them to use proper autolinks
for consistency and improved rendering in documentation tools.

Link: Rust-for-Linux#1153

Signed-off-by: Abhishek Brahmbhatt <brambhattabhishek@gmail.com>
@brambhattabhishek

Copy link
Copy Markdown
Author

please can anyone review this PR

@ojeda

ojeda commented Apr 9, 2025

Copy link
Copy Markdown
Member

@ojeda ojeda force-pushed the rust-next branch 5 times, most recently from bfc3cd8 to a3b2347 Compare May 27, 2025 18:27
@tamird

tamird commented May 30, 2025

Copy link
Copy Markdown
Member

This is obsolete, right?

@ojeda

ojeda commented May 30, 2025

Copy link
Copy Markdown
Member

Yeah, I think so (86d990c ("rust: convert raw URLs to Markdown autolinks in comments")).

@brambhattabhishek is there anything left here? (we don't use PRs anymore here, and there are many outdated ones, but it may help others closing it to avoid confusion). Thanks!

@ojeda ojeda force-pushed the rust-next branch 3 times, most recently from 5d132fa to 3b83f5d Compare October 22, 2025 05:47
@ojeda ojeda force-pushed the rust-next branch 4 times, most recently from 7c678ba to 438700e Compare April 3, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants